✨ PR Review Suggestion

Bug - src/server/server.py (Line 70)

Details:

Problem: Thread creation has incorrect args format - should be tuple
Fix: Add comma to make single argument a tuple
Why: Without comma, args expects a sequence but gets a single value, causing TypeError

-    newConnectionsThread = threading.Thread(target = newConnections, args = (sock))
+    newConnectionsThread = threading.Thread(target = newConnections, args = (sock,))

Security - src/client/client.py (Line 18)

Details:

Problem: Bare except clause catches all exceptions including KeyboardInterrupt
Fix: Specify socket.error exception type
Why: Catching specific exceptions improves error handling and allows clean program termination

-        except:
+        except socket.error:

Security - src/server/server.py (Line 34)

Details:

Problem: Bare except clause catches all exceptions including KeyboardInterrupt
Fix: Specify socket.error exception type
Why: Catching specific exceptions improves error handling and allows clean program termination

-            except:
+            except socket.error:

This review was generated by LinearB AI and Added by gitStream